Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix compile warnings/error reported in Windows #3616

Merged
merged 2 commits into from
Jul 12, 2024
Merged

Fix compile warnings/error reported in Windows #3616

merged 2 commits into from
Jul 12, 2024

Conversation

wenyongh
Copy link
Contributor

@wenyongh wenyongh commented Jul 11, 2024 •
edited
Loading

Clear some compile warnings and fix undefined reference error for symbol ffs
in Windows platform.

yamt
yamt reviewed Jul 11, 2024
View reviewed changes
core/iwasm/aot/aot_runtime.c Show resolved Hide resolved
core/iwasm/common/wasm_loader_common.c Show resolved Hide resolved
core/iwasm/compilation/aot_llvm.c Outdated Show resolved Hide resolved
@TianlongLiang
Copy link
Contributor

LGTM

@wenyongh wenyongh merged commit 73caf19 into bytecodealliance:main Jul 12, 2024
378 checks passed
@wenyongh wenyongh deleted the clear_compile_warnings branch July 15, 2024 00:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yamt yamt yamt left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wenyongh @TianlongLiang @yamt